Skip to content

fix: require a comment with every thumbs-down - #350

Merged
AtrikGhosh merged 2 commits into
dvafrom
fix/thumbs-down-requires-comment
Nov 12, 2026
Merged

AtrikGhosh merged 2 commits into
dvafrom
fix/thumbs-down-requires-comment

Conversation

@mimran-khan

@mimran-khan mimran-khan commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

What

Fixes #351

How

Testing

  • Unit tests added/updated
  • Ran locally (npm run dev)
  • Manual verification (describe below if applicable)

Rollback

Checklist

  • PR title follows Conventional Commits (feat:, fix:, ci:, etc.)
  • No secrets, credentials, or PII in the diff
  • No breaking changes (or documented above with a migration path)
  • Lint and tests pass (npm run lint && npm run test)

@mimran-khan
mimran-khan requested a review from a team as a code owner November 11, 2026 19:00
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@mimran-khan
mimran-khan force-pushed the fix/thumbs-down-requires-comment branch from 2115e2c to 5805f8f Compare November 11, 2026 19:02
@mimran-khan mimran-khan linked an issue Nov 11, 2026 that may be closed by this pull request
@mimran-khan mimran-khan linked an issue Nov 11, 2026 that may be closed by this pull request
A down vote could be sent with an empty comment. Send stays disabled until the comment has text, and Enter does not submit a blank one.

Signed-off-by: mimran-khan <mohammed_imran.khan@outlook.com>
An interrupt with no answer text was treated as a dropped stream, so the
run stayed loading and the approval never resumed. A paused interrupt now
ends the stream, and the auto-approve test reports the thread as interrupted.

Signed-off-by: mimran-khan <mohammed_imran.khan@outlook.com>
@mimran-khan
mimran-khan force-pushed the fix/thumbs-down-requires-comment branch from 5805f8f to 658a499 Compare November 12, 2026 07:03
@AtrikGhosh
AtrikGhosh merged commit facbde8 into dva Nov 12, 2026
12 checks passed
@AtrikGhosh
AtrikGhosh deleted the fix/thumbs-down-requires-comment branch November 12, 2026 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Require a comment on thumbs-down

2 participants